Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reword availability quiz for clarity. Fixes #162 #164

Merged
merged 3 commits into from
Sep 4, 2024

Conversation

david-a-wheeler
Copy link
Contributor

No description provided.

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
@david-a-wheeler
Copy link
Contributor Author

@mkdolan - is this better?

@david-a-wheeler
Copy link
Contributor Author

@SecurityCRob - any thoughts on this reword?

Copy link

@SecurityCRob SecurityCRob left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

line 394 - remove "commercial". EVERYONE cares about availability, enterprise or not. Also, change the listed attackers to "threat actors" or "malicious actors". Not everything is organized crime or nation-state. "Script kiddies" are some of the likeliest threats to availability.
line 402 - don't change to "guarantee". That implies something like a contract. Ensure is the better word

@david-a-wheeler
Copy link
Contributor Author

@SecurityCRob - great points, thank you.

I'll remove "commercial" and changed "guarantee" to ensure".

Regarding the listed attackers - you're right, that's too specific. Let's make it even simpler, let's just eliminate that part of the text. All that matters is being the "target of a highly-resourced distributed denial-of-service attack (DDoS)" - it really doesn't matter what their motivations are.

I'll post the update. If no other comments come in, we can just merge.

Clarify availability question.

Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
Signed-off-by: David A. Wheeler <dwheeler@dwheeler.com>
@david-a-wheeler david-a-wheeler merged commit d4b32f3 into main Sep 4, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants